Skip to content

Update interface with OpenCLEvaluation#8

Merged
gsvgit merged 4 commits intoSparseLinearAlgebra:devfrom
dpanfilyonok:simple-bfs-benchmark
Jan 18, 2021
Merged

Update interface with OpenCLEvaluation#8
gsvgit merged 4 commits intoSparseLinearAlgebra:devfrom
dpanfilyonok:simple-bfs-benchmark

Conversation

@dpanfilyonok
Copy link
Copy Markdown
Collaborator

@dpanfilyonok dpanfilyonok commented Jan 15, 2021

Proposed Changes

  • Interface updated
  • Fody disabled all over solution

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Additional comments

@dpanfilyonok dpanfilyonok requested a review from gsvgit January 15, 2021 16:34
[<AutoOpen>]
module BFS =
let levelBFS (matrix: Matrix<bool>) (source: int) : Vector<int> =
let levelBFS (matrix: Matrix<bool>) (source: int) : OpenCLEvaluation<Vector<int>> =
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can it be naturally generalized to multisource BFS?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but at the moment we still won't be able to test it. These algorithms are given as an example of use and may not be completely correct.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok.

@gsvgit gsvgit merged commit e5c9ed5 into SparseLinearAlgebra:dev Jan 18, 2021
@dpanfilyonok dpanfilyonok deleted the simple-bfs-benchmark branch February 15, 2021 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants